-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tooling #21
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @steven-murray! I'll get tests up and running once we've merged this in.
I just have a couple of small comments.
@steven-murray looks like there are some merge conflicts that need to be resolved as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @steven-murray!
This adds a number of new tooling/package restructure things:
src/
setup.cfg
instead ofsetup.py
primarilyflake8
checks and plugins to both pre-commit and toxtests/
directory and TRAVIS toolingdocs/
directory and skeleton, but no actual docsThe main thing to be checked here are the
flake8
plugins and options -- whether we want all these (or more!).Addresses #21
Addresses #11